Skip to content

Use pinned Hermes version when version.properties is not 1000.0.0#56607

Merged
cipolleschi merged 4 commits into0.85-stablefrom
fix/hermes-version-085
Apr 24, 2026
Merged

Use pinned Hermes version when version.properties is not 1000.0.0#56607
cipolleschi merged 4 commits into0.85-stablefrom
fix/hermes-version-085

Conversation

@cipolleschi
Copy link
Copy Markdown
Contributor

@cipolleschi cipolleschi commented Apr 24, 2026

Summary

  • Reads HERMES_VERSION_NAME / HERMES_V1_VERSION_NAME from version.properties to decide whether to use nightly Hermes, instead of relying on the branch name.
  • Only runs use-hermes-nightly.js or sets latest-v1 when the version is 1000.0.0 (the development placeholder).
  • Fixes incorrect nightly Hermes usage on PR branches targeting stable.

This is the 0.85-stable backport of the same fix applied to 0.83-stable.

Changelog:

[Internal] - Fix CI

Test plan

  • CI should pass on this PR with the pinned Hermes version from version.properties rather than falling back to nightly.

The use-hermes-nightly flag was computed from the branch name, which
evaluates incorrectly for PR branches targeting stable. Instead, read
the actual HERMES_VERSION_NAME from version.properties and only use
nightly when it is the 1000.0.0 development placeholder.
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 24, 2026
@github-actions
Copy link
Copy Markdown

Warning

JavaScript API change detected

This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API.

  • Please include a clear changelog message.
  • This change will be subject to additional review.

This change was flagged as: POTENTIALLY_BREAKING

The Hermes version is now derived from version.properties directly,
so the use-hermes-nightly input and all caller references are no
longer needed.
The Maven Central artifact version uses the hermes-v prefix (e.g.
hermes-v250829098.0.10), but the version.properties file stores
the raw version number. Add the prefix when constructing the
HERMES_VERSION env var so the prebuild script finds the artifact.
The Maven Central artifact uses the raw version number (e.g.
250829098.0.10), not the hermes-v prefixed form.
@cipolleschi cipolleschi merged commit 5e3a3ba into 0.85-stable Apr 24, 2026
49 checks passed
@cipolleschi cipolleschi deleted the fix/hermes-version-085 branch April 26, 2026 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant